Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

부산대 Android_김주송_4주차 과제 (STEP2) #86

Open
wants to merge 62 commits into
base: jooiss
Choose a base branch
from
Open

부산대 Android_김주송_4주차 과제 (STEP2) #86

wants to merge 62 commits into from

Conversation

jooiss
Copy link

@jooiss jooiss commented Jul 19, 2024 •
edited
Loading

코드 작성 시, 어려웠던 점

  • UI 테스트 코드는 처음 작성하다보니 제대로 작성한 것이 맞는지에 대한 의문이 계속 들었습니다.
  • View와 관련되어있지만 함수의 기능을 검증하는 테스트 코드는 기능 테스트 파일에 넣어야 할 지, UI 테스트 파일에 넣어야 할 지 고민이 되었습니다. (함수 기능을 검증하는데 초점을 맞췄다고 생각하여 일단 기능 테스트 파일에 첨부해놨습니다.)

중점적으로 리뷰해주셨으면 하는 부분

  • thread.sleep()으로 검색 대기 시간을 설정했는데 그 안에 넣어주는 시간에 따라 테스트 결과가 달라지는데 테스트 코드가 잘못 설계되어 있어 그런건지 아니면 대기 시간 차이에 의한 오류인지 궁금합니다.
  • 장소를 검색하는 PlaceActivity 상에서의 화면 UI를 테스트하는 코드에서 안드로이드 키보드가 영어로 설정되어 있으면 한국어 입력이 안되는 현상을 발견하였습니다. 이를 제어하기 위한 다른 라이브러리나 방법이 있는지 궁금합니다
  • 테스트 코드 사용법이 미숙해서 최대한 레퍼런스들을 참고하여 진행하였는데 제가 사용한 방법이 실제로 많이 사용하는 방법인지, 아니면 더 효율적으로 똑같은 테스트 단위를 확인할 수 있는 방법이 있는지에 대해 궁금합니다.

테스트 성공 결과

  • UI 테스트 코드
    image image
  • Unit 테스트 코드
    image image

항상 정성스러운 리뷰 감사합니다. 단계를 일정 안에 수행하지 못해서 죄송합니다. 다음부터 빨리 진행하여 일정 상 차질이 없도록 하겠습니다. 감사합니다.

MyStoryG and others added 30 commits June 25, 2024 01:19
@MyStoryG
Initial commit
04c907f
@jooiss
docs: Update README.md
eeebf44
@jooiss
feat: Design search layout
c032344
@jooiss
feat: Add place database
f011ead 
- PlaceContract : 테이블 이름 및 열 이름 정의
- PlaceDatabaseHelper : 데이터베이스 생성
- PlaceDatabaseAccess : 데이터베이스에 데이터 저장 및 삭제 기능
@jooiss
fix: Add trigger to prevent storing duplicate data
a289932
@jooiss
feat: Design search recyclerView and place recyclerView
121176b 
- place_item : place list item layout
- search_item : search list item layout
@jooiss
feat: Set the layout to be displayed
05af938 
- PlaceDataModel : Place 데이터 클래스
- PlaceRecyclerViewAcitivity : Place 목록을 보여줄 recycler view 제어
- PlaceRecyclerViewAdapter : Place 목록 adapter
@jooiss
docs: Update README.md
166b988 
- STEP2 구현 기능 목록 추가
@jooiss
feat: Add search function of place list
34627be
@jooiss
refactor: change place list item
ca0170c
@jooiss
refactor: resize search list item layout
400b36f
@jooiss
feat: Add storing function of record list
cf1cfe3
@MyStoryG
Initial commit
bbf77e9
@jooiss
fix: Resolve conflict
e0916b7
@jooiss
fix: Exclude kakao map dependency
df4cb5e
@jooiss
docs: Update README.md
c86d1d0
@jooiss
feat: Add database insert function
3242db7
@jooiss
fix: Resolve screen-off feature
fd929f2
@jooiss
feat: Set API key value for kakao API integration
d50c57e
@jooiss
feat: Add data transfer object file
93f3d31
@jooiss
feat: Add retrofit interface
3b25cb4
@jooiss
refactor: Combine data activity and place activity
0ae951c
@jooiss
feat: Add retrofit object
3d9c0a1
@jooiss
feat: Update network security configuration
55aef9e
@jooiss
feat: Add database insertion function
7381db6
@jooiss
refactor: Extract string to file
e52f182
@jooiss
refactor: Change the search item design
bf0301d
@jooiss
feat: Reset the data at first
ac3b1a5
@jooiss
feat: Change to category search
67f2174
@jooiss
docs: Update README.md
bb02709
jooiss added 17 commits July 12, 2024 21:08
@jooiss
fix: Make API key resource not be saved
4cf9b86
@jooiss
fix: Call MapView method in time for activity
68f9aa8
@jooiss
fix: Change lifecycle function about kakao map API
51f84de
@jooiss
fix: Change CategoryGroup finding method
36b98d5
@jooiss
refactor: Separate the Retrofit network-related function
17351fa
@jooiss
fix: Resolve conflict
858022d
@jooiss
docs: Update README.md
1b66d98
@jooiss
refactor: Delete unnecessary files
37bad8f
@jooiss
feat: Change search method by keyword
34e1c44
@jooiss
feat: Display search result on selecting a saved search record
4f84a78
@jooiss
feat: Display modal when clicking on place list
4ef142d
@jooiss
feat: Show error message when onMapError
7cd48c3
@jooiss
feat: Resume from last position on restart
23e8fb5
@jooiss
test: Add UI test code for PlaceActivity
072cae8
@jooiss
test: Add UI test code for MapActivity
caeef42
@jooiss
test: Add unit test code for MapActivity
fa1b72e
@jooiss
test: Add unit test code for PlaceActivity
96d0b32
@LeeOhHyung LeeOhHyung self-requested a review July 21, 2024 18:52
LeeOhHyung
LeeOhHyung approved these changes Jul 21, 2024
View reviewed changes
Copy link

@LeeOhHyung LeeOhHyung left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM~ 고생많으셨습니다.
별도의 코멘트가 없을만큼 테스트코드도 잘 작성해주셨고, 요구사항도 모두 만족하게 만들어주셨네요.

아쉬운 점이 있다면, UI, network, utils 등등 클래스 목적에 맞춰 패키지를 나눠 보시면 더 좋을것 같습니다. 그리고, 학습했던 MVVM 아키텍처에 맞춰서 필요한 의존성 클래스들 (ex. Repository, UseCase, API, DBHelper) 의 객체를 클래스 내부에서 생성하는게 아닌, 외부에서 만들어서 생성자를 통해 전달받는 구조로 만들어보시면 좀더 좋을것 같습니다.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@LeeOhHyung LeeOhHyung LeeOhHyung approved these changes

Assignees

@jooiss jooiss

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@jooiss @LeeOhHyung @MyStoryG